Skip to content

Fix pressed keys reset when hiding a window on Windows#104802

Merged
Repiteo merged 1 commit into
godotengine:masterfrom
Hilderin:fix-pressed-keys-resetted-when-hiding-window-on-windows
Nov 18, 2025
Merged

Fix pressed keys reset when hiding a window on Windows#104802
Repiteo merged 1 commit into
godotengine:masterfrom
Hilderin:fix-pressed-keys-resetted-when-hiding-window-on-windows

Conversation

@Hilderin
Copy link
Copy Markdown
Contributor

The problem was introduced in #99010 to fix an issue where the user was unable to resize the floating window or use buttons in the editor when the embedded game was activated on Windows. Prior to #99010, the window needed to be focused in order to be resized using the mouse. The modification, however, introduced a side effect: it reset the pressed keys in Input when hiding a window, because deactivation was no longer handled by the timer.

By retaining the timer solely for the deactivation process, the original issue remains resolved, and it also fixes #104479, as the pressed keys are no longer reset immediately when calling the hide method on a window.

@Hilderin Hilderin requested a review from a team as a code owner March 30, 2025 14:49
@Hilderin Hilderin added this to the 4.5 milestone Mar 30, 2025
@Repiteo Repiteo modified the milestones: 4.5, 4.6 Sep 5, 2025
@Repiteo Repiteo added the cherrypick:4.5 Considered for cherry-picking into a future 4.5.x release label Sep 5, 2025
@akien-mga akien-mga requested a review from bruvzg October 7, 2025 10:42
Comment thread platform/windows/display_server_windows.cpp Outdated
@Repiteo Repiteo force-pushed the fix-pressed-keys-resetted-when-hiding-window-on-windows branch from e39f6d4 to f323337 Compare November 18, 2025 16:26
@AThousandShips AThousandShips changed the title Fix pressed keys resetted when hiding a window on Windows Fix pressed keys reset when hiding a window on Windows Nov 18, 2025
@Repiteo
Copy link
Copy Markdown
Contributor

Repiteo commented Nov 18, 2025

Rebased to include @bruvzg's suggestion

@Repiteo Repiteo merged commit 76835d2 into godotengine:master Nov 18, 2025
20 checks passed
@Repiteo
Copy link
Copy Markdown
Contributor

Repiteo commented Nov 18, 2025

Thanks!

@akien-mga
Copy link
Copy Markdown
Member

Cherry-picked for 4.5.2.

@akien-mga akien-mga removed the cherrypick:4.5 Considered for cherry-picking into a future 4.5.x release label Jan 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Hold shift to add both Environment and Sun to scene no longer works on 4.4

4 participants